Skip to content

fix(schema): align doctor JSON schema with Cargo installer field#230

Open
cursor[bot] wants to merge 1 commit into
mainfrom
cursor/critical-bug-inspection-1b51
Open

fix(schema): align doctor JSON schema with Cargo installer field#230
cursor[bot] wants to merge 1 commit into
mainfrom
cursor/critical-bug-inspection-1b51

Conversation

@cursor
Copy link
Copy Markdown
Contributor

@cursor cursor Bot commented May 10, 2026

Summary

Cargo 支持已在 quantex doctor --jsondata.installers 中增加 cargo 字段,但 quantex schema doctor 返回的 JSON Schema 仍只声明 brew/bun/npm/winget,且在 installers 上使用 additionalProperties: false。任何按该 schema 严格校验真实 doctor 输出的集成都会误报失败。本 PR 将 cargo 纳入 schema 的 propertiesrequired,并补充测试与 OpenSpec 契约说明。另移除 src/self/index.ts 中与 export from './facts' 重复的导入,消除 oxlint 的未使用导入报错(否则本地/CI 无法通过 bun run lint)。

Linked Artifacts

  • OpenSpec: openspec/changes/fix-doctor-json-schema-cargo-installer

Validation

  • bun run memory:check
  • bun run lint
  • bun run format:check
  • bun run typecheck
  • bun run test (if behavior changed)
  • Not run, explained below

在本环境中通过 ./node_modules/.bin/oxlintoxfmt --checktsc --noEmitvitest run 完成等价校验。memory:check 脚本依赖 Bun 的 import.meta.dir,当前沙箱无 Bun 未执行。

Release Intent

  • Release: patch - bug fix

Docs Updated

  • openspec/...

Scope Check

  • I did not add a new ad hoc root-level Markdown file.
  • I updated the relevant issue, ADR, spec, runbook, or captured the missing doc work as follow-up.
  • I did not silently expand project scope without recording it explicitly.

Closure Check

  • Working tree was clean after commit.
  • Branch was pushed and this PR is the active delivery artifact.
  • OpenSpec change is not needed, still active by design until merge, already archived, or queued for agent-driven archive closure.
  • Release is not applicable, delegated to release automation, or verified.

Notes

无。

Open in Web View Automation 

Add cargo to doctor dataSchema installers so strict schema validation
matches quantex doctor --json. Sync OpenSpec agent-update spec and tests.

Remove redundant self/index facts imports that triggered oxlint unused-import
errors.
@cursor cursor Bot force-pushed the cursor/critical-bug-inspection-1b51 branch from fd6e3af to f97f75e Compare May 13, 2026 02:24
@Drswith Drswith marked this pull request as ready for review May 13, 2026 06:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant